listbox: Don't compare int to uint

colorbutton: Stop using state-changed

toolbar: Limit item position to number of contained elements

Greater values don't make much sense.

https://bugzilla.gnome.org/show_bug.cgi?id=682080

wayland: Avoid unitialized memory reads

I didn't pay attention when I replaced g_new0 with g_newa. Oops.

Adwaita: brighter pushed button background

providing more contrast with the text color.

Adwaita: bold size label on font button

looks more consistent with other special buttons with icons.

Adwaita: hide separator on button.font and button.file

we used to do that, got missed at some point during 3.20 cycle.

Docs: Clarify life cycle issue

gtk_gesture_get_last_event() wasn't very clear about how long
it is safe to use the returned pointer.

DND: Avoid a use-after-free

Cancelling the gesture causes the last_event pointer to become
invalid. Make a copy of the event so we can keep using it
regardless of the gesture state.

places sidebar: Avoid markup confusion in tooltips

At one point, the sidebar was using gtk_treeview_set_tooltip_column,
which expects tooltips to be markup. With the listbox-based sidebar,
we don't do that anymore. So don't escape the tooltip text.

https://bugzilla.gnome.org/show_bug.cgi?id=766175

x11: Trap possible X error

XIGetClientPointer can generate X errors (e.g. when the X server
does not support XI2. Trap them and carry on.

https://bugzilla.gnome.org/show_bug.cgi?id=766233

Fix scale length

The vertical scales should really be of the same length, so
add an empty value to be drawn by the other scale.

Forgotten rename

Scale: Set the correct value alignment

We're measuring both the width of the minimal and the maximal value for
the value gadget, but only give the value the minimal width for its
current value, resultnig in an always left-aligned value.

Fix this by assigning the width of the value gadget to the value layout
and letting pango align the text inside the layout.

https://bugzilla.gnome.org/show_bug.cgi?id=766120

widget-factory: Let one of the scales draw a value

https://bugzilla.gnome.org/show_bug.cgi?id=766120

Document new api

Make gdk_event_get_pointer_emulated public

There is not strong reason to keep the getter private.
At the same time, strip _-prefixes from a few other GdkEvent
APIs. Update all callers.

gtkmenu: ignore left/right scroll events

Menus are placed vertically by definition, it does not make much sense
to support horizontal axis for scrolling.

Use GDK_EVENT_STOP/GDK_EVENT_PROPAGATE instead of TRUE/FALSE and add a
default case to return GDK_EVENT_PROPAGATE for unhandled events.

https://bugzilla.gnome.org/show_bug.cgi?id=765939

gtkmenu: ignore emulated scroll events

On X11, we get both smooth and emulated scroll events, whereas other
backends such as wayland will give smooth events only with touchpad
scrolling.

Discard emulated scroll events so that we get consistent behaviours
between backends.

Allow for both horizontal and vertical smooth events for scrolling so
that horizontal scrolling still works without emulated scroll events as
well, again for consistency between gdk backends.

https://bugzilla.gnome.org/show_bug.cgi?id=765939

gdkevent: make _gdk_event_get_pointer_emulated() private

And not just internal to gdk, so we can use it in gtk as well, to
differentiate emulated scroll events from others.

https://bugzilla.gnome.org/show_bug.cgi?id=765939

scale: Remove double nullable annotation

Being both (allow-none) and (nullable) at the same time is a bit much.
Was from 591e7f5ef8538982e227b2c2cefc536a33cafa6c.

widget: Add Since annotation to gtk_widget_queue_allocate

GTK: Fix build for pre-C99 compilers

Some compilers we support, such as pre-2013 Visual Studio, does not support
for INIFINITY, log2() and exp2(), so check for exp2() and log2() during
configure, and use fallbacks for them and INIFINTY if they are not found.

https://bugzilla.gnome.org/show_bug.cgi?id=766207

Visual Studio builds: Refine "installation" process

Split out the part where we generate/update the caches for the GSchemas
and the icons, so that it is easier to ensure that things continue to
function correctly when we have GlibEtcInstallRoot != CopyDir.

scrolledwindow: Remove child before destroying self

Children tend to call back into the scrolled window while being removed
and that doesn't work too well if the scrolled window is destroyed
already as Christian Hergert found out.

cssimage: Make it possible to shrink builtin check/optionmarks

... below 7px of size.

Updated Occitan translation

docs: Update examples of matching key bindings

The selector for matching GtkEntry has changed to `entry` after 3.20.

https://bugzilla.gnome.org/show_bug.cgi?id=766166

gdk/Makefile.am: Dist gdkmonitorprivate.h

This file needs to be distributed with the tarball as well...

Visual Studio builds: Clean up projects a bit

Consolidate the .lib's that we need to link to for GDK and GTK+ into the
property sheets, and also link to setupapi.lib as that is needed by
the port to the new GdkMonitor API.

gdk/win32/gdkmonitor-win32.c: Fix build on non-GCC

WINBOOL is MinGW-specific, so change it to BOOL, which is universally
available.

Also, Visua Studio is more picky on where __stdcall (WINAPI) is placed, so
fix that to be in-sync with what is done in the other sources.

placesview: update guiding list of protocols

The previous list of protocols in the guidance popover
was suboptimal and needed adjustments.

This commit updates this list to match the latest mockups [1].

[1] https://raw.githubusercontent.com/gnome-design-team/gnome-mockups/4847c79ea1aaf75046b3ec5bb1b167ac604d86ed/nautilus/nautilus-next/connect-to-server-wire.png

https://bugzilla.gnome.org/show_bug.cgi?id=756570

dialogs: remove 2px border width again

Updated Lithuanian translation

Updated Brazilian Portuguese translation
(cherry picked from commit 207b793316a90faff00d5ea6fbe483d5a184d066)

tooltip: Include gtkwidgetprivate.h

That's where gtk_widget_query_tooltip is defined.

gdk: Fix parameter name mismatch

infobar: Fix typo

Style class names are prefixed with a '.'

gtk-demo: Fix revealer demo

This only used by luck before. We are changing a property from the
::notify handler for that property. Now that GtkRevealer is notifying
the property when it stops animations on unmap, we end up in a life
lock situation where we never make it out of the notify queue.

Fix this by not restarting the animation if the widget is unmapped.

widget: add missing detail to ::query-tooltip emission

This was an oversight in commit dfb368e29bb58b4313b578f0ce75cfc8ead9a1.

tooltip: Add a missing *

This was an oversight in the previous commit.

gtk3-demo: Add a close button to the markup window

gtk-demo: Don't duplicate text in markup example

I forgot to clear the buffer before inserting the new markup. Oops.

revealer: emit notify::child-revealed when animation stops

Depending of float rounding during target calculation, the size of the
GtkRevealer can be set to zero will the animation is not finished.
If the GtkRevealer is in a GtkPaned, it will be hidden and so the animation
will be stopped before it is finished.

In this case, force the emission of the child-revealed signal to let
client code know the animation is finished.

https://bugzilla.gnome.org/show_bug.cgi?id=765973

Simplify our templates

This is applying the new property name simplification.

Clean up builder parser data after parsing

No need to have these linger around in qdata.

builder tool: Canonicalize property names

When simplifying, convert property names to canonical form,
so we don't waste time while looking them up later.

Updated Occitan translation

Adwaita: darken treeview exapanders

Adwaita: cosmetic change

Adwaita: relocate a selector

relocate treview acceleditor > label in the treeview section and
add a comment for a testcase.

Avoid emitting ::style-set by name

GtkStyle is deprecated, but we still emit ::style-set quite
a bit, so lets at least not be slow while doing it.

Don't emit ::query-tooltip by name

This signal is emitted quite a bit, and we can easily avoid it.

Stop storing has-tooltip in qdata

This is queried quite a bit, and we have room for an extra
bit in GtkWidgetPrivate.

Don't use qdata for getting the default css style

There's no point in using qdata for looking this up, since
we only ever set it on the default screen anyway.

Adwaita: tidy treeview style properties...

...removing a double definition in the process.

settings: Fix font names *again*

This is the last time, I hope.

widget: Store accessible in GtkWidgetPrivate

Every widget may have one of these, and they are accessed somewhat
frequently.

Ensure we initialize a quark first

The style function here can also be called before the corresponding
class_init had a chance to create the quark. Fix this.

tooltip: Ensure our quark is not 0

Many of the tooltip functions can be called before a tooltip
object is created, and thus initializing the quark in class_init
is not good enough.

Use a cheaper way to store settings per display

This was another very frequent use of qdata. Since we typically
have only one or two display objects, storing the display-settings
association in a simple array is faster than using object data
or a hash table.

Use a cheaper way to store debug flags

This was the most frequent use of qdata *by far*.

Fix indentation mishap

Fix a typo

wayland: use g_signal_handler_disconnect()

Use of g_signal_handlers_disconnect_by_func() needs to do more work than
necessary to find all the matching handlers. Instead, just hold on to the
signal identifier and remove it directly so we hit the fast path.

Not terribly ground breaking in terms of performance gains, but its done
enough to be worthwhile.

https://bugzilla.gnome.org/show_bug.cgi?id=766049

Update color name css style test

Verify rebeccapurple too.

scale: Fix mark positioning at the left/top edge

We were failing to take the widget allocation.x/y into account
when deciding whether we need to push in the mark.

https://bugzilla.gnome.org/show_bug.cgi?id=765922

Generalize the previous commit

Update all xsettings when we get a screen, to prevent similar
problems from occurring in the future.

settings: Avoid a crash

Christian Hergert reported seeing webkit crashes with recent
GTK+. The stacktrace points at the CSS machinery calling into
GtkSettings to get the font name, and then getting surprised
by a property notification that triggers style validation.

To avoid this, query the font name xsetting right away when
we get set a screen.

x11: Rely on passive keyboard grabs with XI2 for DnD

The active keyboard grab can be spared then. This way the passive
key grabs allow other key combinations (eg. alt-tab) that are not
mandatory to grab here.

Adwaita: more flexible headerbar_fill mixin

add an additional background layer as a parameter.

searchenginetracker: Quote fts match so it's seen as a single phrase

FTS5 has a complex enough syntax that it makes not much sense to wrap
in simple search boxes.

https://bugzilla.gnome.org/show_bug.cgi?id=765981

label: Also initialize text

So gtk_label_get_text returns the default value of "". Broke the
GtkModelButton default value test.

cssgadget: Remove unused function

gtkrender: Remove superfluous cairo_save/restore calls

These were all surrounding functions that either call
cairo_save/cairo_restore themselves or don't need to.

css node: Stop extraneous change notification

The ::name and ::widget-type properties are entirely independent,
no need to notify both when either changes.

box: Only emit ::child-notify when necessary

We can easily check if the values for expand, padding and so forth
actually changed, so lets do it.

Revert "When creating a widget path, use the widget type"

This reverts commit 0d78b67bca8866a05d0d55d3d5c48aa7a90ff89e.

As Benjamin points out: that'll break all widgets that query style
properties in their init function.

When creating a widget path, use the widget type

No need to pull the type out of the css node - its our own type.
This will let us stop setting the type on the css node later on.

Set the proper state on the css node

This will almost certainly overwritten before the widget gets
to the screen, but while we are doing this, we might as well
use the same state that we initialize the widgets state to.

css names are always set

No need to check for it, we set the css name on GtkWidgetClass
ourselves.

switch: Remove .slider from slider gadget

As the documentation states, neither the switch nor the slider gadget
should be using any style classes.

switch: Create less pango layouts

Instead of creating 2 pango layouts in every draw() and on in
_get_content_size (and calling into gettext twice in both cases), just
keep the layouts around and create them in only one place.

settings: Fix xsettings handling

I was somehow under the misconception that we'd get GdkEventSettings
events for all the xsettings at startup. That is not in general true,
so we need to make sure that we check for the xsettings value before
we use them, or derived fields. Update all the private getters to
do so; and fix settings_update_font_values() to cope with font
descriptions that might miss the family or size.

settings: Ensure derived fields are initialized

I mistakenly assumed that gtk_settings_init was already doing
something to trigger a notify for all properties. It doesn't,
so we have to ensure that settings_update_font_values() is
called at least once.

https://bugzilla.gnome.org/show_bug.cgi?id=765966

wayland: Small improvements to update_direction

Avoid memory allocation if possible, and use the proper
min/max keycodes.

wayland: Don't emit direction-changed unless it did

We can easily check this, so lets do it.

trivial formatting fixes

Use g_set_object in one more place

label: don't use set_text in gtk_label_new

It would just set up lots of things with the default values, but a
common case is that a few gtk_label_set_* calls follow.

Updated Norwegian bokmål translation.

label: minimum/natural size parameters are not nullable

Stop comparing them to NULL.

label: Don't set out parameters twice

We set them later again when we calculate the vertical size, so just
move the general assignment to -1 into the horizontal branch.

widget: Remove useless NULL check

We are checking for allocation != NULL at the beginning of the function
already. This also means that the allocation parameter is not nullable.

roundedbox: Initialize the hashtable key before using it

In mem_hash, we are iterating over the individual bytes of the key, so
make sure all of it is initialized.

cssimagebuiltin: Remove icons that don't draw anything anymore

cssimagebuiltin: Remove unused variables

widget: Remove useless comment

wayland: fix up/down mix up in discrete events

The wayland specification for discrete step information for scroll and
other axes reads:

| The discrete value carries the directional information. e.g. a
| value of -2 is two steps towards the negative direction of this axis.

mutter sets a value of 1 for SCROLL_DOWN events and -1 for SCROLL_UP
events.

gdkdevice Wayland backend does the opposite, it translates a positive
discrete value as SCROLL_UP and a negative value as SCROLL_DOWN, which
ends up inverting the scrolling direction.

Fix the logic in gdkdevice Wayland to use a positive value as
SCROLL_DOWN and a negative value as SCROLL_UP so that it matches mutter
and weston logic.

https://bugzilla.gnome.org/show_bug.cgi?id=765907

css: Stop drawing double borders for builtins

The fallback code for rendering builtin checks, radios
and expanders was using border parameters. With the generic
gadget borders using the same parameters, this was giving
double borders.